Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Take care about mariadb platform #742

Merged
merged 1 commit into from
Dec 24, 2014

Conversation

nlegoff
Copy link
Contributor

@nlegoff nlegoff commented Dec 12, 2014

Hi,
After upgrading to DBAL 2.5, I got an issue where I could not rename index while migrating because of MariaDB versioning which outputs 10.0.15-MariaDB-1~wheezy as server version.

Because 10.x > 5.7 it loads new features from mysql 5.7 which are not available in mariadb ..

@doctrinebot
Copy link

Hello,

thank you for creating this pull request. I have automatically opened an issue
on our Jira Bug Tracker for you. See the issue link:

http://www.doctrine-project.org/jira/browse/DBAL-1073

We use Jira to track the state of pull requests and the versions they got
included in.

@deeky666
Copy link
Member

looks sane to me. Please add some test data here and I'll merge.

@deeky666 deeky666 self-assigned this Dec 13, 2014
@Ocramius
Copy link
Member

Should we have a different platform for mariadb? The entire mariadb-mysql-yaddayadda-galeracluster-xtradb stuff is going to get out of hand.

@nlegoff nlegoff force-pushed the mysql-version-detection branch from 39911d8 to 9a7a4a0 Compare December 15, 2014 10:23
@nlegoff nlegoff force-pushed the mysql-version-detection branch from 9a7a4a0 to 7f89a78 Compare December 15, 2014 10:28
@nlegoff
Copy link
Contributor Author

nlegoff commented Dec 15, 2014

@deeky666 added some test cases.

@stof
Copy link
Member

stof commented Dec 19, 2014

@Ocramius having a specific platform which would be the same than MySQLPlatform is not worth the maintenance IMO. And having a MariaDBDriver forcing the usage of this platform would make things more complex.

IMO, detecting MariaDB versions is the best approach for our users, as this keeps MariaDB as a drop-in replacement (but it is a drop-in replacement for 5.6 in its version 10.0)

@Ocramius
Copy link
Member

The point is that MariaDB is becoming less of a drop-in replacement as time passes (like with any forked software)

@deeky666
Copy link
Member

@stof @Ocramius I tend to agree with you both. I think this solution here is acceptable for now considering, that the current DBAL MySQL platforms are not differenciated enough to even reflect all of MySQL's version specific semantics (especially reserved keywords which even tend to change in patch versions regularly).
Introducing distinct platforms for MariaDB would also require to add a distinct schema manager, tests etc. This is a lot of effort for nearly no benefit in the current state. Also I don't remember we mention anywhere that MariaDB is officially supported by DBAL. So we should reconsider differenciating both vendors in 3.0 but I doubt it makes a lot of sense in 2.x.

@Ocramius
Copy link
Member

I doubt it makes a lot of sense in 2.x.

👍

deeky666 added a commit that referenced this pull request Dec 24, 2014
@deeky666 deeky666 merged commit 19f5a13 into doctrine:master Dec 24, 2014
@deeky666
Copy link
Member

@nlegoff thx!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants